Skip to content

[DOC] Doc for class String#893

Closed
BurdetteLamar wants to merge 2 commits into
ruby:masterfrom
BurdetteLamar:json_string_doc
Closed

[DOC] Doc for class String#893
BurdetteLamar wants to merge 2 commits into
ruby:masterfrom
BurdetteLamar:json_string_doc

Conversation

@BurdetteLamar

@BurdetteLamar BurdetteLamar commented Nov 12, 2025

Copy link
Copy Markdown
Member

Text for String class doc, for String.json_create, and for String#to_json_raw.

:nodoc: for String#to_json_raw_object, which I don't think users need.

@byroot byroot left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for this.

However this is an API that I do intend to extract in an optional gem, because it's very dangerous an inefficient. So I'm not sure it really needs much advertisement.

Comment thread lib/json/add/string.rb Outdated
Co-authored-by: Jean Boussier <jean.boussier@gmail.com>
@BurdetteLamar BurdetteLamar requested a review from byroot November 12, 2025 16:45
@BurdetteLamar BurdetteLamar closed this by deleting the head repository Jan 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants